Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(megamenu): fix alignment issues in mobile nav #5112

Merged

Conversation

annawen1
Copy link
Member

@annawen1 annawen1 commented Feb 5, 2021

Related Ticket(s)

React & Web component: Mega Menu - Hamburger menu options are not left aligned and divider lines are missing. #5079
Web component: Mega Menu - On landscape mode, IBM logo is cut off and last few mega menu options are not visible. #5080

Description

BEFORE:
Screen Shot 2021-02-05 at 4 18 47 PM

AFTER:
Screen Shot 2021-02-05 at 3 56 45 PM

Changelog

Changed

  • set border-bottom and height for submenus
  • set text to show ellipsis when overflow in menu item links

@annawen1 annawen1 added package: react Work necessary for the Carbon for IBM.com react components package package: web components Work necessary for the IBM.com Library web components package labels Feb 5, 2021
@ibmdotcom-bot
Copy link
Contributor

@ibmdotcom-bot
Copy link
Contributor

@ibmdotcom-bot
Copy link
Contributor

Copy link
Member

@jeffchew jeffchew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@annawen1 annawen1 requested a review from kennylam February 8, 2021 14:00
@annawen1 annawen1 added the Ready to merge Label for the pull requests that are ready to merge label Feb 8, 2021
@kodiakhq kodiakhq bot merged commit 8cd3f1e into carbon-design-system:master Feb 8, 2021
@annawen1 annawen1 deleted the fix/megamenu-update-fixes branch February 10, 2021 18:19
IgnacioBecerra pushed a commit to IgnacioBecerra/ibm-dotcom-library that referenced this pull request Feb 22, 2021
…em#5112)

### Related Ticket(s)

React & Web component: Mega Menu - Hamburger menu options are not left aligned and divider lines are missing. carbon-design-system#5079

### Description

BEFORE:
<img width="409" alt="Screen Shot 2021-02-05 at 4 18 47 PM" src="https://user-images.githubusercontent.com/54281166/107090920-eb16dc80-67ce-11eb-87d1-05b025535dda.png">

AFTER:
<img width="415" alt="Screen Shot 2021-02-05 at 3 56 45 PM" src="https://user-images.githubusercontent.com/54281166/107090929-ee11cd00-67ce-11eb-9bda-3370155643bc.png">

### Changelog
**Changed**

- set border-bottom and height for submenus
- set text to show ellipsis when overflow in menu item links

<!-- React and Web Component deploy previews are enabled by default. -->
<!-- To enable additional available deploy previews, apply the following -->
<!-- labels for the corresponding package: -->
<!-- *** "package: services": Services -->
<!-- *** "package: utilities": Utilities -->
<!-- *** "package: styles": Carbon Expressive -->
<!-- *** "RTL": React / Web Components (RTL) -->
<!-- *** "feature flag": React / Web Components (experimental) -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: react Work necessary for the Carbon for IBM.com react components package package: web components Work necessary for the IBM.com Library web components package Ready to merge Label for the pull requests that are ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants